Skip to content

fix(gateway): materialize the WaveKV winner in proxy state - #1067

Merged
kvinwang merged 1 commit into
nextfrom
fix/proxy-state-wavekv-winner
Aug 18, 2026
Merged

fix(gateway): materialize the WaveKV winner in proxy state#1067
kvinwang merged 1 commit into
nextfrom
fix/proxy-state-wavekv-winner

Conversation

@kvinwang

Copy link
Copy Markdown
Collaborator

Summary

  • treat the value selected by WaveKV as authoritative during ProxyState reloads
  • remove the second value-level LWW decision based on reg_time
  • preserve runtime connection counters and clean up replaced IP/app indexes
  • add a regression test for a WaveKV winner with an older payload timestamp

Testing

  • cargo test -p dstack-gateway (158 passed)
  • cargo fmt --all --check
  • git diff --check
  • cargo clippy -p dstack-gateway --all-targets -- -D warnings (blocked by pre-existing clippy::assertions_on_constants in gateway/src/web_routes/wavekv_sync.rs:567 on next)

Copilot AI lite review requested due to automatic review settings August 18, 2026 02:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kvinwang
kvinwang force-pushed the fix/proxy-state-wavekv-winner branch from bba15c1 to 1963f48 Compare August 18, 2026 02:31
@kvinwang
kvinwang merged commit 7f36d78 into next Aug 18, 2026
16 checks passed
@kvinwang
kvinwang deleted the fix/proxy-state-wavekv-winner branch August 18, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants